Skip to content

Remove ERROR from parse_rules#78

Merged
FreddieAkeroyd merged 2 commits intomasterfrom
rerpha-patch-1
May 5, 2026
Merged

Remove ERROR from parse_rules#78
FreddieAkeroyd merged 2 commits intomasterfrom
rerpha-patch-1

Conversation

@rerpha
Copy link
Copy Markdown
Contributor

@rerpha rerpha commented Apr 29, 2026

ie. https://epics-jenkins.isis.rl.ac.uk/job/InstrumentChecker/3447/parsed_console/

where FAILED is the only actual useful information and the process returns 1 anyway if it falls over.

@rerpha rerpha added this to Tasks Apr 29, 2026
@rerpha rerpha moved this to Flash Review in Tasks Apr 29, 2026
@FreddieAkeroyd
Copy link
Copy Markdown
Member

Might it be worth matching FAIL [ instead of FAILED so we get a link to each failure?

@FreddieAkeroyd FreddieAkeroyd self-requested a review April 30, 2026 14:46
@FreddieAkeroyd FreddieAkeroyd merged commit 0ceac21 into master May 5, 2026
4 checks passed
@FreddieAkeroyd FreddieAkeroyd deleted the rerpha-patch-1 branch May 5, 2026 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants